Skip to content

Introduce OuterZkvmHost trait and wire SP1 aggregation into parallel prover#2724

Merged
bkolad merged 8 commits intodevfrom
blaze/agg_proof_19
Apr 15, 2026
Merged

Introduce OuterZkvmHost trait and wire SP1 aggregation into parallel prover#2724
bkolad merged 8 commits intodevfrom
blaze/agg_proof_19

Conversation

@bkolad
Copy link
Copy Markdown
Member

@bkolad bkolad commented Apr 13, 2026

Description

This PR introduces a dedicated outer-proof host path for aggregated proofs and wires it through the prover stack.

Changes:

  1. Add OuterZkvmHost and Zkvm::OuterHost for host-side proof aggregation.
  2. Change create_aggregated_proof to take DA block headers instead of slot hashes.
  3. Thread full DA headers through zk_manager / prover services so outer provers can consume header data directly.
  4. Implement outer-host support for:
    • MockZkvmHost
    • SP1AggregationHost
    • Risc0Host as unimplemented!() for now
  5. Update SP1 demo/test wiring so the parallel prover uses SP1 for both inner and outer proofs.
  6. Refresh integration and demo tests to use block headers for aggregation requests.

Notes

  • The aggregation circuit currently has a temporary NOMT workaround: the state-root continuity assertion is commented out with a TODO.

  • The SP1 aggregation host now keeps previous outer-proof state internally so recursive aggregation can continue across calls.

  • I have updated CHANGELOG.md with a new entry if my PR makes any breaking changes or fixes a bug. If my PR removes a feature or changes its behavior, I provide help for users on how to migrate to the new behavior.

  • I have carefully reviewed all my Cargo.toml changes before opening the PRs. (Are all new dependencies necessary? Is any module dependency leaked into the full-node (hint: it shouldn't)?)

Linked Issues

@bkolad bkolad force-pushed the blaze/agg_proof_19 branch 4 times, most recently from e457f52 to 51feb26 Compare April 14, 2026 08:14
@bkolad bkolad force-pushed the blaze/agg_proof_19 branch from 51feb26 to 525e2e0 Compare April 14, 2026 08:19
@bkolad bkolad force-pushed the blaze/agg_proof_19 branch from 836d597 to 67e7fa3 Compare April 15, 2026 10:40
@bkolad bkolad changed the title WIP agg proof Introduce OuterZkvmHost trait and wire SP1 aggregation into parallel prover Apr 15, 2026
Comment thread crates/adapters/sp1/src/host.rs
@bkolad bkolad marked this pull request as ready for review April 15, 2026 11:41
@bkolad bkolad changed the title Introduce OuterZkvmHost trait and wire SP1 aggregation into parallel prover Introduce OuterZkvmHost trait and wire SP1 aggregation into parallel prover Apr 15, 2026
@bkolad bkolad merged commit d62d3f8 into dev Apr 15, 2026
24 of 26 checks passed
@bkolad bkolad deleted the blaze/agg_proof_19 branch April 15, 2026 13:15
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants